(PC-34083)[API] fix: pro: draft offer: always check if EAN is used #15965
Conversation
|
Visit the preview URL for this PR (updated for commit 7ec9126): https://pc-pro-testing--pr15965-pc-34083-block-pendi-i9lv1ws4.web.app (expires Thu, 30 Jan 2025 16:56:06 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1 |
jbaudet-pass
force-pushed
the
PC-34083/block_pending_offer_update_if_published_offer_with_same_ean_exists
branch
from
3b9f3e8 to
13184c1
Compare
jbaudet-pass
had a problem deploying
to
pullrequest
GitHub Actions
Failure
jbaudet-pass
force-pushed
the
PC-34083/block_pending_offer_update_if_published_offer_with_same_ean_exists
branch
2 times, most recently
from
100ff2a to
7ec9126
Compare
Before publishing or updating a draft offer, the EAN should always be checked: if it is already used by another published (managed by the same venue), the edit or updated operation should fail.
jbaudet-pass
had a problem deploying
to
pullrequest
GitHub Actions
Failure
| class DraftOfferFactory(OfferFactory): | ||
| isActive = False | ||
|
|
There was a problem hiding this comment.
Je crois qu'il faut que tu rajoutes cela pour que ta factory soit complète.
| class DraftOfferFactory(OfferFactory): | |
| isActive = False | |
| class DraftOfferFactory(OfferFactory): | |
| isActive = False | |
| validation = OfferValidationStatus.DRAFT | |
| if offer.extraData: | ||
| if ean := offer.extraData.get(ExtraDataFieldEnum.EAN.value): | ||
| validation.check_other_offer_with_ean_does_not_exist(ean, offer.venue, offer.id) | ||
|
|
There was a problem hiding this comment.
Pour l'update je pense que tu peux enlever cette partie. Effectivement dans certain cas le check ne sera pas parfait (si on n'a pas changé les "extraData" et que pourtant entre temps une offre a été publiée avec l'ean qui était dans l'extraData existant) mais ce n'est pas si grave.
Le plus important c'est de bloquer la publication de l'offre.
But de la pull request
Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-34083
Avant
Sur le portail pro, il était possible d'éditer en de publier une offre en brouillon même si l'EAN était déjà utilisé par une offre publiée.
Pour l'offre en brouillon, il faut la créer, en publier une avec le même EAN et revenir dessus : il n'y a plus de vérification sur l'EAN.
Après
La vérification au niveau de l'EAN se fait au moment de la publication et de l'édition d'un brouillon d'offre, et plus seulement à sa création.